COMP: Build against ITK main; sync ExternalData config; fix docs env#456
Open
hjmjohnson wants to merge 3 commits into
Open
COMP: Build against ITK main; sync ExternalData config; fix docs env#456hjmjohnson wants to merge 3 commits into
hjmjohnson wants to merge 3 commits into
Conversation
Superbuild ITK_TAG v5.4.2 -> main, matching the C++ CI jobs and picking up post-5.4.2 fixes behind the Python baseline mismatches. pip itk pins -> >=5.4.6; zarr>=3.0.0 satisfies ngff-zarr OME-Zarr 0.5 (nbsphinx notebook execution failed without it). ExternalData URL templates and module now mirror ITK main: adds the Girder hashsum mirror (sole host of some legacy sha512 objects, e.g. ComputeFFTInOneDimension), filebase and pinata public gateways, and the WASI symlink guard; drops dead estuary.tech/w3s.link and the released-root-CID gateways. The project-specific gin.g-node.org mirror and ExternalData_LINK_CONTENT=CID are retained.
Last remaining hash-based links in the repository; blobs published under CID in ITKTestingData#64.
github-actions
Bot
added
area:Filtering
Member
Author
|
Finding from #455's re-run: the sha512→cid conversion here is required, not cosmetic — the top-level |
Member
Author
|
Folded the visual-cid fix from #455 (closed) into this PR as f759c48 — with ITKTestingData#63/#64/#65 all merged, every content link this PR's CI needs is now published. Remaining known blocker: the ITKReview/zlib link regression on ITK main hits the superbuild jobs (fix validated locally; ITK PR pending). |
github-actions
Bot
added
area:Core
The 20 PNGs referenced since InsightSoftwareConsortium#440 were never published to ITKTestingData and are unrecoverable from any public source; every build of Module_SphinxExamples fails 20 ExternalData fetches. Point the .cid files at recreated diagrams (ITKTestingData#63) and fix the Operator.png/Operation.png reference mismatch.
hjmjohnson
force-pushed
the
fix/ci-itk-pin-and-deps
branch
from
f759c48 to
d2521d3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Build against ITK main, sync the ExternalData configuration with ITK upstream, fix the docs notebook env (
zarr>=3), and finish the repo's sha512→cid conversion. With ITKTestingData#63/#64 merged, this addresses the remaining CI-infrastructure failures on every open PR. Blocked on ITKTestingData#65 (CID copies of the FFT blobs).What each change fixes
ITK_TAGv5.4.2 →main(matches the C++ jobs'itk-git-tag)itk>=5.4.2/>=5.4rc3→>=5.4.6zarr>=3.0.0in the superbuild venvbuild_htmlhard failure: nbsphinx executingConvertNumPyArrayToitkImage.ipynbhitsValueError: zarr-python >= 3.0.0b2 required for OME-Zarr >= 0.5ITKSphinxExamplesExternalData.cmake+ExternalData.cmakesynced to ITK mainExternalData_LINK_CONTENT=CIDretainedComputeFFTInOneDimension.sha512→.cid(last 6 hash links repo-wide)MouseLiverRF.mha does not exist); blobs published in ITKTestingData#64 (SHA512) and #65 (CID)